Skip to content

test: cover host context binding precedence#2602

Merged
chubes4 merged 1 commit into
mainfrom
fix/issue-2592-agents-api-context-bindings
Jun 8, 2026
Merged

test: cover host context binding precedence#2602
chubes4 merged 1 commit into
mainfrom
fix/issue-2592-agents-api-context-bindings

Conversation

@chubes4

@chubes4 chubes4 commented Jun 8, 2026

Copy link
Copy Markdown
Member

Summary

  • Tighten the Data Machine-to-Agents API context handoff so top-level host payload fields win over same-named client_context keys.
  • Add focused smoke/unit assertions for explicit client_context_bindings reading authoritative host payload values.

Follow-up to #2594 after a self-review pass on #2592.

Tests

  • php tests/tool-source-registry-smoke.php passed: all 63 assertions.
  • php -l inc/Engine/AI/Tools/ToolExecutor.php passed.
  • php -l tests/tool-source-registry-smoke.php passed.
  • php -l tests/Unit/AI/Tools/ToolExecutorValidationTest.php passed.

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (openai/gpt-5.5)
  • Used for: Performed a self-review pass, identified the host/context precedence edge case, implemented the small follow-up, and ran focused verification. Chris remains responsible for review and testing.

@chubes4 chubes4 merged commit b8306f6 into main Jun 8, 2026
5 checks passed
@chubes4 chubes4 deleted the fix/issue-2592-agents-api-context-bindings branch June 8, 2026 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant